9주차미션_윈트 #17
Open
9주차미션_윈트 #17
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
chrkb1569
approved these changes
May 28, 2023
There was a problem hiding this comment.
코드량이 상당했는데 중간중간 주석이 있어서 어떠한 역할을 수행하는 파일인지 확인하기가 수월하였습니다!
저도 이번 미션에서 JWT를 통한 로그인 기능을 구현하였으나, 로그아웃 기능은 구현하지 않아 구현하신 기능에 대하여 자세히 보았는데, redis를 활용하여 로그아웃 여부를 확인할 수 있을 줄은 몰랐네요..
또한, querydsl은 듣기만 했지 실제로는 어떻게 동작하는지 잘 몰랐던 기술이기에, 이번 미션을 리뷰하면서 많이 알아가는 것 같습니다.
마지막 application-prod.yml 파일의 경우에는 s3에 접근할 수 있는 키값들이 노출되어 있는데,
보안적인 문제로 인하여 .gitignore 를 사용해 노출을 피하시는 게 좋을 것 같습니다.
이번 9주차 미션 하시느라 고생 많으셨습니다!!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
작업 내용 👨🏻💻
jwt를 이용한 인증인가를 추가하였습니다.
To Reviewers 💬
Reference 🔬